Support parsing of special characters in TypeToSchema visitor #220

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

rdblue merged 7 commits into apache:master from rdsr:special_chars

Jun 22, 2019

Contributor

rdsr commented Jun 15, 2019

This addresses #216

rdsr commented

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java Outdated Show resolved Hide resolved

rdblue reviewed

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java Outdated Show resolved Hide resolved

rdblue reviewed

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java Outdated Show resolved Hide resolved

rdblue reviewed

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java Outdated Show resolved Hide resolved

rdblue reviewed

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java Outdated Show resolved Hide resolved

rdblue reviewed

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/AvroSchemaUtil.java Outdated Show resolved Hide resolved

rdblue reviewed

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java Outdated Show resolved Hide resolved

rdblue reviewed

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java Outdated Show resolved Hide resolved

rdblue reviewed

View reviewed changes

core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java Outdated Show resolved Hide resolved

rdsr added 7 commits

June 21, 2019 14:42


          Support parsing of special characters in TypeToSchema visitor

eae5e22


          Address comments


          Fix typo

25473e8


          Address comments

a229cb0


          Update test case

c2140db


          Address comment and improve testcase

ad88fa6


          Address comments

54f8b43

rdsr force-pushed the special_chars branch from a8cb8a6 to 54f8b43 Compare

June 21, 2019 23:04

rdblue merged commit 930ebc7 into apache:master

andrei-ionescu mentioned this pull request

Fix Iceberg Reader for nested partitions (#575) #585

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet